Skip to content

Fix Farcaster web share embed rendering#603

Merged
realproject7 merged 1 commit intomainfrom
task/598-share-embed-fix
Mar 27, 2026
Merged

Fix Farcaster web share embed rendering#603
realproject7 merged 1 commit intomainfrom
task/598-share-embed-fix

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

Fixes #598

  • Farcaster web compose fallback now uses embeds[] query parameter for the story URL instead of appending it to the text body
  • Enables proper link preview/embed rendering when sharing via web
  • Miniapp SDK composeCast path unchanged (already uses embeds array)
  • X/Twitter share unchanged (URL in text body is the standard pattern)

Test plan

  • Share to Farcaster via web → story URL renders as embed preview, not inline text
  • Share to Farcaster via miniapp → still works (SDK path unchanged)
  • Share to X → URL still appears in tweet text
  • Build passes

🤖 Generated with Claude Code

Use separate embeds[] query parameter for the story URL in the
Farcaster web compose fallback instead of embedding the URL in the
text body. This enables proper link preview/embed rendering on
Farcaster. Miniapp SDK path unchanged. X/Twitter share unchanged.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
plotlink Ignored Ignored Mar 27, 2026 4:33pm

Request Review

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE

Summary

The PR is correctly scoped to issue #598 and fixes the Farcaster web fallback by moving the story URL into the embeds[] parameter while leaving the miniapp SDK path unchanged.

Findings

  • None.

Decision

This satisfies the requested Farcaster web share fix without introducing unrelated changes.

@realproject7 realproject7 merged commit 62ce1e3 into main Mar 27, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix trailing whitespace in share text breaking OG image and miniapp rendering

2 participants